Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 📦 Provide RudderStack Chart with HPA #44

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

afranzi
Copy link

@afranzi afranzi commented Feb 12, 2022

Description of the change

This PR aims to provide the RudderStack helm as a Chart while providing the following features:

  • Feature: Provide Chart structure.
  • Feature: Release Chart on master merge.
  • Feature: Move (on demand) Postgres resources to a sidecar approach.
  • Feature: Enable HPA for the Transformations Deployment resource.
  • Feature: Enable HPA for the Backend Statefulset resource.
  • Feature: PreStop Hook to prevent data loss on new deployments or autoscaling down processes.
  • Fix: backend.uploadFreqInS typo.
  • Feature: Add env tag to Telegraf global tags.
  • Feature: Allow to define extraEnvVars in the Telegraf pod.

⚠️ Note: It requires the RudderStack team to enable gh-pages to host the Chart.

Related issues

PR #43

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests
  • All tests related to the changed code pass in development

Code review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
  • "Ready for review" label attached to the PR and reviewers mentioned in a comment
  • Changes have been reviewed by at least one other engineer
  • Issue from task tracker has a link to this pull request

afranzi and others added 16 commits January 31, 2022 18:47
* feat: 🐳 - Add GHA to publish helm

* Set theme jekyll-theme-time-machine

* Set theme jekyll-theme-cayman

* feat: 🐳 - Add GHA to publish helm

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* refactor: 📦 - Create subcharts & proper chart folder

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* fix: 📦 Move templates folder under the Chart one

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* fix: 📦 Add dependencies

* fix: 📦 Add dependencies

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* fix: 📦 Fix if identations

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* feat: 📝 - Allow to provide external config

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* feat: 🔒 - Allow to define the ServiceAccount

* Update index.yaml

Signed-off-by: afranzi <[email protected]>

* feat: 🔒 - Fix ServiceAccount position

* feat: 🔒 - Bump version to 0.4.5

* Prepare PR

* feat: [DAT-3878] - Allow to define extra env vars in telegraf

* feat: [DAT-3878] - Bump version to 0.4.6

* feat: [DAT-3936] 🔧 Move postgresql from statefulset to sidecar

* feat: [DAT-3936] 🔧 Add missing env var

* feat: [DAT-3936] 🔧 Fix rediness command

* feat: [DAT-3936] 🔧 Bump version

* feat: [DAT-3936] 🔧 Fix postgres host

* feat: [DAT-3936] 🔧 Remove postgres subchart

* feat: [DAT-3936] 🔧 Bump version to 0.4.11

* doc: [DAT-3898] - 📝 Update Documentation and add missing postgres values

* feat: [DAT-3850] 🔧 Allow to define the terminationGracePeriodSeconds value

* feat: [DAT-3850] - 💄 Cleaning values

* feat: [DAT-3850] 📦 Put postgres back

* fix [DAT-3850] - Define COMPUTE_DB_HOST_IN_K8S

* fix [DAT-3850] - Define ephemeral volume

Co-authored-by: afranzi <[email protected]>
* feat [DAT-4136] - 🔧 Provide a pre-stop script

* feat [DAT-4136] - 🔧 Solve script path
* feat [DAT-4136] - 🔧 Emit statsd pre-stop hook points

* feat [DAT-4136] - 🚀 Bump Chart version to v0.4.3
@tf-seti tf-seti deleted the branch rudderlabs:master December 19, 2023 11:24
@tf-seti tf-seti deleted the master branch December 19, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants